Skip to content

Expose more fields in OperatingSystemStatus #199

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
toanju merged 1 commit into from
Dec 17, 2025
Merged

Expose more fields in OperatingSystemStatus #199

toanju merged 1 commit into from
Dec 17, 2025

Conversation

@toanju
Copy link
Contributor

@toanju toanju commented Dec 16, 2025 •
edited
Loading

Expose more fields in OperatingSystemStatus

Exposes the following additional fields

  • GardenLinuxCommitID
  • GardenLinuxFeatures
  • VariantID

to share more details on a Garden Linux based installation

@toanju toanju requested review from fwiesel and notandy December 16, 2025 09:02
@fwiesel
Copy link
Contributor

fwiesel commented Dec 16, 2025 •
edited
Loading

I see the need in it, and it is probably just bike-shedding, but this is tied to the gardenlinux build process, and doesn't cover all the inputs (i.e. GARDENLINUX_CNAME).

How about setting and using some standard fields instead?
I'd be a fan of setting BUILD_ID to ${{github.workflow}}-${{github.run_number}} or ${{github.run_id}}.

Or would VARIANT_ID and GARDENLINUX_COMMIT_ID suffice to uniquely identify the build?

fwiesel
fwiesel reviewed Dec 16, 2025
View reviewed changes
Copy link
Contributor

@fwiesel fwiesel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll approve it, in case you want to move forward regardless of the alternatives proposed.
Worst-case we migrate..

@fwiesel
Copy link
Contributor

fwiesel commented Dec 16, 2025 •
edited
Loading

Small detail, I'd go a least with commitID or even gardenlinuxCommitID as that is the name in the os-release file.

@toanju
Copy link
Contributor Author

toanju commented Dec 16, 2025 •
edited
Loading

we were discussing if BUILD_ID actually suites here, however BUILD_ID does not really match for Garden Linux since we do not have a particular underlying build that resides the same. Will update to commitID.

notandy
notandy requested changes Dec 16, 2025
View reviewed changes
@toanju toanju changed the title Expose commit sha of Garden Linux images Expose more fields in OperatingSystemStatus Dec 17, 2025
@toanju toanju requested review from fwiesel and notandy December 17, 2025 08:30
@toanju
Copy link
Contributor Author

toanju commented Dec 17, 2025

I agree with

would VARIANT_ID and GARDENLINUX_COMMIT_ID suffice to uniquely identify the build

and adjusted accordingly and added the Features in addition

anokfireball
anokfireball approved these changes Dec 17, 2025
View reviewed changes
Copy link

@anokfireball anokfireball left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it is worth it to "properly" expose the feature list? Don't feel forced to follow through with this suggestion, though!

@toanju
Copy link
Contributor Author

toanju commented Dec 17, 2025

yes, I was thinking about the list, but still checking on the guarantees here

@toanju
Copy link
Contributor Author

toanju commented Dec 17, 2025

list it is...

notandy
notandy requested changes Dec 17, 2025
View reviewed changes
@toanju
Expose more fields in OperatingSystemStatus
7faabc9 
Exposes the following additional fields

* GardenLinuxCommitID
* GardenLinuxFeatures
* VariantID

to share more details on a Garden Linux based installation
@github-actions
Copy link

github-actions bot commented Dec 17, 2025

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/cobaltcore-dev/openstack-hypervisor-operator/api/v1 0.00% (ø)
github.com/cobaltcore-dev/openstack-hypervisor-operator/applyconfigurations/api/v1 0.00% (ø)
Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/cobaltcore-dev/openstack-hypervisor-operator/api/v1/hypervisor_types.go 0.00% (ø) 0 0 0
github.com/cobaltcore-dev/openstack-hypervisor-operator/api/v1/zz_generated.deepcopy.go 0.00% (ø) 0 0 0
github.com/cobaltcore-dev/openstack-hypervisor-operator/applyconfigurations/api/v1/operatingsystemstatus.go 0.00% (ø) 0 0 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

@toanju toanju merged commit b801e15 into main Dec 17, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@notandy notandy notandy approved these changes

@fwiesel fwiesel fwiesel approved these changes

@anokfireball anokfireball anokfireball approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@toanju @fwiesel @notandy @anokfireball